Skip to content

Annotate array_namespace and add py.typed #254

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 27, 2025

Conversation

NeilGirdhar
Copy link
Contributor

This appears to be the only missing type annotation in the project.

This appears to be the only missing type annotation in the project.
@NeilGirdhar
Copy link
Contributor Author

Eventually, Namespace can point to Namespace ultimately exported by array-api-typing.

@ev-br ev-br closed this Feb 21, 2025
@ev-br ev-br reopened this Feb 21, 2025
@NeilGirdhar
Copy link
Contributor Author

Any chance we could get this in before the next release?

image

@ev-br ev-br closed this Feb 26, 2025
@ev-br ev-br reopened this Feb 26, 2025
@ev-br
Copy link
Member

ev-br commented Feb 27, 2025

Okay, CI is green after a rerun, let's get this in. Thanks @NeilGirdhar , thanks @crusaderky

@ev-br ev-br merged commit 61c83fe into data-apis:main Feb 27, 2025
83 of 120 checks passed
@NeilGirdhar
Copy link
Contributor Author

Thank you!

@NeilGirdhar NeilGirdhar deleted the annotations branch February 27, 2025 11:53
@crusaderky
Copy link
Contributor

It's worth pointing out that the addition of py.typed will cause all type annotations to be picked up by downstream modules, and I've spotted many issues with them.
I'm writing a follow-up PR to remediate.

@ev-br
Copy link
Member

ev-br commented Feb 27, 2025

PSA: I am literally in the release process, and will carve it once gh-250 is green.
So if it's disruptive or needs checking, let's back please it up, so that we can fix things up at leisure.

@NeilGirdhar
Copy link
Contributor Author

@crusaderky Awesome, thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants